Skip to content

fix (core): Update workflow border radius. - #2691

Open
khathija-ahamadi wants to merge 2 commits into
mainfrom
fix/4309-mismatch-figma-workflow-border
Open

fix (core): Update workflow border radius.#2691
khathija-ahamadi wants to merge 2 commits into
mainfrom
fix/4309-mismatch-figma-workflow-border

Conversation

@khathija-ahamadi

@khathija-ahamadi khathija-ahamadi commented Aug 3, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

💡 What is the current behavior?

workflow-steps dont have border radius in code but they do in figma

🆕 What is the new behavior?

Border radius is updated for the workflow-step component.

🏁 Checklist

A pull request can only be merged if all of these conditions are met (where applicable):

  • 🦮 Accessibility (a11y) features were implemented
  • 🗺️ Internationalization (i18n) - no hard coded strings
  • 📲 Responsiveness - components handle viewport changes and content overflow gracefully
  • 📕 Add or update a Storybook story
  • 📄 Documentation was reviewed/updated siemens/ix-docs
  • 🧪 Unit tests were added/updated and pass (pnpm test)
  • 📸 Visual regression tests were added/updated and pass (Guide)
  • 🧐 Static code analysis passes (pnpm lint)
  • 🏗️ Successful compilation (pnpm build, changes pushed)

👨‍💻 Help & support

Summary by CodeRabbit

  • Style
    • Updated workflow component corners to use a consistent 0.25rem border radius for a more standardized appearance.

@changeset-bot

changeset-bot Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1fcbf4a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify

netlify Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for ix-storybook ready!

Name Link
🔨 Latest commit 1fcbf4a
🔍 Latest deploy log https://app.netlify.com/projects/ix-storybook/deploys/6a7426d7d229950008933310
😎 Deploy Preview https://deploy-preview-2691--ix-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 0b723b10-2a37-4c18-9098-2db5342b6e6b

📥 Commits

Reviewing files that changed from the base of the PR and between 60d38b5 and 47818bd.

📒 Files selected for processing (1)
  • packages/core/scss/theme/core/components/workflow.scss
📝 Walkthrough

Walkthrough

The workflow theme border-radius variable changes from 1px to 0.25rem.

Changes

Workflow theme

Layer / File(s) Summary
Update workflow border radius
packages/core/scss/theme/core/components/workflow.scss		 The workflow border-radius variable now uses 0.25rem instead of 1px.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the workflow border-radius update, which is the main change in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/4309-mismatch-figma-workflow-border

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@khathija-ahamadi
khathija-ahamadi marked this pull request as ready for review August 3, 2026 14:29
@khathija-ahamadi khathija-ahamadi changed the title Update workflow border radius. fix (core): Update workflow border radius. Aug 3, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JonaMaiss JonaMaiss JonaMaiss approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@khathija-ahamadi @JonaMaiss